Skip to content

feat(sandbox): Landlock TCP port restriction for mandatory proxy#16

Open
Ladas wants to merge 1 commit into
feat/platform-modefrom
feat/landlock-tcp-port
Open

feat(sandbox): Landlock TCP port restriction for mandatory proxy#16
Ladas wants to merge 1 commit into
feat/platform-modefrom
feat/landlock-tcp-port

Conversation

@Ladas

@Ladas Ladas commented Jun 12, 2026

Copy link
Copy Markdown

Summary

Add Landlock ABI v4 TCP port restriction to Platform mode. Restricts
connect() to proxy port only (3128). All other ports return EACCES
at the kernel level. Makes the CONNECT proxy mandatory, not cooperative.

Depends on: #15 (Platform mode base)

1 file, +43/-3 lines. Compiles, tests pass, clippy clean.

Ref: NVIDIA#899

Assisted-By: Claude Code

@Ladas Ladas force-pushed the feat/platform-mode branch from 421a0a7 to 869b3f0 Compare June 12, 2026 16:25
@Ladas Ladas force-pushed the feat/landlock-tcp-port branch from 9ec5718 to 179d108 Compare June 12, 2026 16:26
Ladas added a commit that referenced this pull request Jun 12, 2026
Add kernel-level network syscall interception using SECCOMP_RET_USER_NOTIF
for Platform mode. Provides mandatory, syscall-level enforcement without
any capabilities.

DnsPinnedAllowlist: resolve domains to IPs at sandbox creation, freeze
for session lifetime (DNS rebinding prevention).

BPF filter intercepts: connect, sendto, sendmsg, recvfrom, recvmsg,
bind. Validates AUDIT_ARCH to prevent x32/compat ABI bypass.

Linux syscall wrappers: notification fd ioctls, pidfd_open/pidfd_getfd
for on-behalf-of operations (TOCTOU-safe), read_process_memory with
read_exact (no short reads), sockaddr parser (correct endianness for
sa_family, port, flowinfo), verify_socket_fd (mitigates fd-swap race),
deny/allow_connect response helpers.

Code review fixes applied across all PRs:
- PR #15: gateway propagates network_enforcement to DriverSandboxSpec
- PR #15: driver uses typed enum comparison (not magic integer)
- PR #16: saturating_sub prevents underflow in Landlock skipped count
- PR #16: warn!() on TCP port restriction failure (was debug)
- PR #17: BPF arch check, recvfrom/recvmsg/bind interception,
  verify_socket_fd, read_exact, allow_connect rename, flowinfo
  endianness, safety comments on all unsafe blocks

8 tests. Compiles, 949 tests pass, clippy clean.

Ref: NVIDIA#899
@Ladas Ladas force-pushed the feat/platform-mode branch from 869b3f0 to ec5655e Compare June 16, 2026 12:43
@Ladas Ladas force-pushed the feat/landlock-tcp-port branch from 179d108 to 59d148a Compare June 16, 2026 14:06
@Ladas Ladas force-pushed the feat/platform-mode branch from ec5655e to 7158b3d Compare June 16, 2026 14:55
When Platform mode is active, apply Landlock ABI v4 network rules to
restrict TCP connect to only the proxy port (default 3128). This makes
the loopback CONNECT proxy mandatory at the kernel level.

Graceful degradation: if kernel ABI < v4, the network rules are skipped
with a warn-level log. Also fixes rules_applied underflow via
saturating_sub.

Ref: NVIDIA#899
Signed-off-by: Ladislav Smola <lsmola@redhat.com>
@Ladas Ladas force-pushed the feat/landlock-tcp-port branch from 59d148a to 8354d68 Compare June 16, 2026 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant